Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Docs: Add plugin security development guide #1195

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Docs: Add plugin security development guide #1195

wants to merge 1 commit into from

Conversation

josmperez
Copy link
Contributor

Draft of new plugin security guide for collaboration with David Harris and others.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 9, 2024

Hello! 👋 This repository uses Auto for releasing packages using PR labels.

❌ This PR cannot be merged until the following issues are addressed:

  • This PR is missing one of the following labels: patch, minor, major, no-changelog.
  • Optionally if using a patch, minor or major label also add the release label if you would like this PR to trigger npm package publishing.
🏷️ More info about which labels to use
  • If the changes only affect the docs website, documentation, or this repository's tooling add the no-changelog label.
  • If there are changes to any of the npm packages src files please choose from one of the following labels:
    • 🐛 if this PR fixes a bug add the patch label
    • 🚀 if this PR includes an enhancement add the minor label
    • 💥 if this PR includes a breaking change add the major label
  • Optionally, if you would like this PR to publish new versions of packages when it is merged add the release label.

irenerl24
irenerl24 approved these changes Oct 10, 2024
View reviewed changes
Copy link

@irenerl24 irenerl24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great doc, LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@irenerl24 irenerl24 irenerl24 approved these changes

@sympatheticmoose sympatheticmoose Awaiting requested review from sympatheticmoose

Assignees

@sympatheticmoose sympatheticmoose

@josmperez josmperez

Labels
None yet
Projects
Plugins Platform / Grafana Community
Status: 🔬 In review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@josmperez @irenerl24 @sympatheticmoose